Skip to content
This repository has been archived by the owner on Nov 22, 2021. It is now read-only.

[WIP] New: implement --init #5

Closed
wants to merge 2 commits into from
Closed

[WIP] New: implement --init #5

wants to merge 2 commits into from

Conversation

mysticatea
Copy link
Member

@mysticatea mysticatea commented Dec 1, 2017

Fixes #4.

This PR adds the feature eslint --init into eslint-cli.

People can set up ESLint for their project with eslint --init command. The command installs eslint and necessary plugins into the project local.

However, eslint-cli requires local-installed ESLint to use eslint --init. This is inconvenient.

# Do `eslint --init` in order to install ESLint locally!
$ eslint --init
Cannot find local ESLint!
Please install ESLint by `npm install eslint --save-dev`.

This PR solves the problem.
Users can use eslint --init in order to install ESLint locally.

Also, this should help us that we remove --init feature from ESLint core to reduce core size in future.


I did refactoring this feature because it has some complex and huge functions.
This is file structure: https://github.com/mysticatea/eslint-cli/tree/init/lib/init

@aladdin-add
Copy link
Member

This package is no longer necessary. Use the built-in npx command instead.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement eslint --init
2 participants